-
Notifications
You must be signed in to change notification settings - Fork 25
HID service unit tests #368
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
ebcd976 to
556f5c1
Compare
|
You can find the documentation preview for this PR here. |
556f5c1 to
d6d717e
Compare
d6d717e to
911a287
Compare
911a287 to
fc0ef29
Compare
57251d0 to
d39b06b
Compare
| evt_rw_auth = &ble_evt.evt.gatts_evt.params.authorize_request; | ||
|
|
||
| switch (auth_evt_type) { | ||
| case ON_OTHER_TYPE: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why the parenthesis? We usually tend to omit them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will remove.
| TEST_ASSERT_EQUAL(NRF_ERROR_NULL, err); | ||
| } | ||
|
|
||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra newline
| /* Exit Suspend command. */ | ||
| #define HIDS_CONTROL_POINT_EXIT_SUSPEND 0x01 | ||
|
|
||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra newline.
6d83510 to
1f501e8
Compare
Add unit tests for ble_hids service. Signed-off-by: Eivind Jølsgard <[email protected]>
1f501e8 to
8383997
Compare
No description provided.